auto merge of #527 : alexcrichton/cargo/build-issues, r=brson

Related issues in the commits.

auto merge of #516 : alexcrichton/cargo/issue-512, r=brson

Closes #512

auto merge of #515 : alexcrichton/cargo/issue-472, r=brson

But still show it with --verbose

Closes #473

auto merge of #522 : EduardoBautista/cargo/update-docs, r=alexcrichton

This PR will make the guide more generic in terms of directory paths and crate information.  It also
updates some outdated output.

auto merge of #513 : bkoropoff/cargo/revert-static, r=alexcrichton

This reverts a commit that worked around an overly restrictive lifetime bound on the `Writer` impl for `Box<Writer>` in libstd.  Now that this restriction has been lifted, the workaround can be reverted.

Don't run cargo as root during `make install`

This rejiggers the dependencies for `make install` to only copy files into the
destination, never assemble anything. This adds a hard requirement that `make`
is executed before `make install`.

Closes #519

Add a note about required build tools and packages

Closes #518

Update output to match latest cargo build

Make docs more generic

Remove an outdated module

Closes #512

Don't show `Fresh` by default

But still show it with --verbose

Closes #473

auto merge of #503 : alexcrichton/cargo/issue-493, r=brson

This commit updates git2-rs to get the implementation of the authentication
callback in libgit2. Additionally this specifies the callback for whenever we're
cloning into the database or updating submodules.

Currently cargo will *not* ask for user input, but rather require you to have
authentication configured in git through some other means. There are currently
two primary methods of doing so:

1. Any SSH key in the local ssh-agent will be used for authentication with SSH
   repositories.
2. The `credential.helper` interface (as specified by gitcredential(7)) has been
   implemented in git2-rs to allow for picking up of storage of passwords in the
   local git cache or keychain.

If these two methods fail, then there will likely be an authentication failure.
Interactive prompts for authentication have not been implemented as there is no
method to currently enter your password into the terminal silently.

A consequence of this commit is that cargo now depends on libssh2. A package was
created to create a static copy of libssh2, and this is now linked into cargo by
default.

It turned out that just building libssh2 was quite a beast in and of itself on
windows. The primary stickler point is that on the current release, 1.4.3,
libssh2 requires openssl on windows. At this time I don't want to pick up a
dependency on openssl for windows, and it turned out that the unreleased 1.4.4
version has a new backend for windows not based on openssl, but rather windows's
cryptography API.

The current bundled version of libssh2 is 1.4.4 with some light modifications to
actually build on windows (wow that was hard). All in all, we're now statically
linking to libssh 1.4.4 (not a runtime dependency).

Closes #493

Implement git authentication

This commit updates git2-rs to get the implementation of the authentication
callback in libgit2. Additionally this specifies the callback for whenever we're
cloning into the database or updating submodules.

Currently cargo will *not* ask for user input, but rather require you to have
authentication configured in git through some other means. There are currently
two primary methods of doing so:

1. Any SSH key in the local ssh-agent will be used for authentication with SSH
   repositories.
2. The `credential.helper` interface (as specified by gitcredential(7)) has been
   implemented in git2-rs to allow for picking up of storage of passwords in the
   local git cache or keychain.

If these two methods fail, then there will likely be an authentication failure.
Interactive prompts for authentication have not been implemented as there is no
method to currently enter your password into the terminal silently.

A consequence of this commit is that cargo now depends on libssh2. A package was
created to create a static copy of libssh2, and this is now linked into cargo by
default.

It turned out that just building libssh2 was quite a beast in and of itself on
windows. The primary stickler point is that on the current release, 1.4.3,
libssh2 requires openssl on windows. At this time I don't want to pick up a
dependency on openssl for windows, and it turned out that the unreleased 1.4.4
version has a new backend for windows not based on openssl, but rather windows's
cryptography API.

The current bundled version of libssh2 is 1.4.4 with some light modifications to
actually build on windows (wow that was hard). All in all, we're now statically
linking to libssh 1.4.4 (not a runtime dependency).

Closes #493

Add back fix for cross compile tests

This change was accidentally included in the reverted commit.

Revert "Be compatible with the stdlib for now ('static shell)"

This reverts commit a601d049feff5768b0d99b9fcec807b277fa4c81.

auto merge of #505 : suhr/cargo/cargo-travis, r=alexcrichton

This implements https://github.com/rust-lang/cargo/issues/489 and fast-fixes https://github.com/rust-lang/cargo/issues/504.

PS: why &* thing is now explicit? This change is rather surprising for me.

Merge with upstream and update Cargo.lock

Conflicts:
src/bin/new.rs
src/cargo/ops/cargo_new.rs

cargo/new: add a test for the `--travis` flag

Revert "cargo/new: add test for `--travis flag; also update Cargo.lock"

This reverts commit 4ee5fd352a0071f4200fe9f3dfa4aeebfe164c18.

auto merge of #508 : carllerche/cargo/static-lib-and-bin-bug, r=alexcrichton

rustc cannot take .a files, so don't pass them

Do not pass static lib targets to rustc

auto merge of #474 : alexcrichton/cargo/issue-461, r=wycats

Overrides are only queried by name, and it's possible for multiple to show up,
and just pick the first one.

Closes #461

r? @wycats -- I'm not convinced this is correct

auto merge of #470 : alexcrichton/cargo/cargo-new-git, r=brson

This adds a command-line --no-git option to disable this behavior, as well as
adding a global config section for `git = false`. While I was at it I write some
documentation for the configuration format that cargo uses.

Turn --git on by default for `cargo-new`.

This adds a command-line --no-git option to disable this behavior, as well as
adding a global config section for `git = false`. While I was at it I write some
documentation for the configuration format that cargo uses.

auto merge of #507 : alexcrichton/cargo/profiles, r=brson

For documentation, see the included documentation in the commit.

Re-enable passing -g to rustc

Now that there is a way to disable debuginfo for a build, we can go back to
passing it by default. Any bugs in debuginfo will get weeded out by specifying
`debug = false` in the profile.

Implement manifest profiles

For documentation, see the included documentation in the commit.

cargo/new: add test for `--travis flag; also update Cargo.lock

auto merge of #506 : alexcrichton/cargo/sooner-lockfile, r=wycats

It's quite annoying if you update a dependency, but it takes you awhile to get
the dependency building. Previously the dependency graph would have to be
updated each time because the lockfile was only written *after* a successful
build.

Other tools like `cargo generate-lockfile` will already generate a lockfile at
any time, so just make it easier by moving it up in the compilation process.

Generate a lockfile sooner in `cargo build`

It's quite annoying if you update a dependency, but it takes you awhile to get
the dependency building. Previously the dependency graph would have to be
updated each time because the lockfile was only written *after* a successful
build.

Other tools like `cargo generate-lockfile` will already generate a lockfile at
any time, so just make it easier by moving it up in the compilation process.

ffix build on nightly rust

cargo/new: add `--travis` flag

auto merge of #501 : alexcrichton/cargo/issue-480, r=brson

If a repository is corrupt, blow it away entirely and start from scratch instead
of returning an error.

Closes #480

auto merge of #500 : alexcrichton/cargo/fix-flaky-test, r=brson

This test has been flaky on the bots for quite some time now, and the cause has
now been discovered. The root cause of the failure is that the execve for the
`cargo --list` command was failing with ETXTBUSY. In querying the manpage, this
means:

    Executable was open for writing by one or more processes.

This error can be explained by the following trace:

1. Thread A, running the `cargo --list` test, opens the destination executable
   for writing because it's copying the current executable into a different
   location.
2. Thread B, some other test, forks the process. The file descriptor of the
   destination executable of thread A is now duplicated in this process.
3. Thread A closes all files and such, and then goes to fork/exec
   `cargo --list`.
4. Thread B has not had time to close all its descriptors. so it still has the
   executable open for writing, causing the `execve` of thread A to fail.

This commit just removes these tested portions of the test, only testing that
cargo probes PATH.

auto merge of #499 : alexcrichton/cargo/issue-486, r=brson

They're not really related to benchmarks!

Closes #486

auto merge of #495 : alexcrichton/cargo/issue-492, r=brson

Closes #492

auto merge of #475 : alexcrichton/cargo/issue-458, r=brson

Relative paths are now considered relative to the directory containing the
`.cargo/config` file specifying the relative path.

I also merge ConfigValueValue into ConfigValue by moving the Path directly next
to the String that defined it so we can track which strings came from which
paths.

Closes #458

auto merge of #491 : alexcrichton/cargo/issue-481, r=brson

The return value of `err.output()` will be `None` if the rustdoc executable
failed to spawn, in which case we can give a more graceful error message.

Closes #481

auto merge of #471 : alexcrichton/cargo/issue-449, r=brson

This is often useful for picking up things like headers files from `*-sys`
packages when they had to compile locally (or perhaps for pkg-config).

Closes #449

auto merge of #469 : alexcrichton/cargo/issue-442, r=brson

Discovering the prefix/suffix needs to understand that it's not actually
available.

Closes #442

Pass native output directories of dependencies

This is often useful for picking up things like headers files from `*-sys`
packages when they had to compile locally (or perhaps for pkg-config).

Closes #449

Fix relative override paths

Relative paths are now considered relative to the directory containing the
`.cargo/config` file specifying the relative path.

I also merge ConfigValueValue into ConfigValue by moving the Path directly next
to the String that defined it so we can track which strings came from which
paths.

Closes #458

Don't fail!() when rustdoc fails to run

The return value of `err.output()` will be `None` if the rustdoc executable
failed to spawn, in which case we can give a more graceful error message.

Closes #481

auto merge of #468 : alexcrichton/cargo/issue-404, r=brson

Whenever possible, try to print a short path by using path_relative_from.

Closes #404

auto merge of #502 : alexcrichton/cargo/try-to-fix-travis, r=alexcrichton

Fix the 32-bit linux travis bots

Slim down the `cargo --list` test

This test has been flaky on the bots for quite some time now, and the cause has
now been discovered. The root cause of the failure is that the execve for the
`cargo --list` command was failing with ETXTBUSY. In querying the manpage, this
means:

    Executable was open for writing by one or more processes.

This error can be explained by the following trace:

1. Thread A, running the `cargo --list` test, opens the destination executable
   for writing because it's copying the current executable into a different
   location.
2. Thread B, some other test, forks the process. The file descriptor of the
   destination executable of thread A is now duplicated in this process.
3. Thread A closes all files and such, and then goes to fork/exec
   `cargo --list`.
4. Thread B has not had time to close all its descriptors. so it still has the
   executable open for writing, causing the `execve` of thread A to fail.

This commit just removes these tested portions of the test, only testing that
cargo probes PATH.

Don't run doc tests for `cargo bench`

They're not really related to benchmarks!

Closes #486

Update docopt to fix -- option parsing

Closes #492

Dedup overrides by name

Overrides are only queried by name, and it's possible for multiple to show up,
and just pick the first one.

Closes #461

Support cross compiling to triples without dylibs

Discovering the prefix/suffix needs to understand that it's not actually
available.

Closes #442

Print longer paths to Cargo.toml on failure

Whenever possible, try to print a short path by using path_relative_from.

Closes #404

Blow away corrupt database repos

If a repository is corrupt, blow it away entirely and start from scratch instead
of returning an error.

Don't use bundled fonts

auto merge of #487 : Insanity133/cargo/patch-2, r=alexcrichton

This is a fixed version of PR #478 on the gh-pages branch.

Fix a minor typo

auto merge of #485 : alexcrichton/cargo/fix-for-rustc-master, r=alexcrichton

Extending #479 to use a `'static` shell instead of a `'a` shell.

Update git2

This updates git2-rs to not use pkg-config on windows.

Closes #472

Be compatible with the stdlib for now ('static shell)

Refresh Cargo.lock

This brings in versions of upstream components with build break fixes

Fix build breakage due to generalized type parameter and object bounds

It is worth noting that cargo was actually creating trait boxes
that contain non-static references, so I couldn't just put 'static
everywhere to appease rustc.

Fix deprecated method/function warnings with rustc master

auto merge of #416 : alexcrichton/cargo/dox, r=wycats

Closes #414

auto merge of #465 : alexcrichton/cargo/update-git2, r=wycats

It turned out most of the methods in libgit2 don't actually require a Signature
structure, they're all mostly optional. This commit updates to this version of
libgit2 where the arguments are all optional.

Closes #463

Update the git2 dependency

It turned out most of the methods in libgit2 don't actually require a Signature
structure, they're all mostly optional. This commit updates to this version of
libgit2 where the arguments are all optional.

auto merge of #459 : alexcrichton/cargo/update-git, r=wycats

An explanation can be found in the commit I made to git2-rs [1]. This is the
first step towards dealing with #457.

[1]: https://github.com/alexcrichton/git2-rs/commit/16142ef9

Link to openssl statically for nightlies

An explanation can be found in the commit I made to git2-rs [1]. This is the
first step towards dealing with #457.

[1]: https://github.com/alexcrichton/git2-rs/commit/16142ef9

auto merge of #453 : dotdash/cargo/glob, r=alexcrichton

The glob crate that comes with rust itself has been deprecated.

auto merge of #455 : alexcrichton/cargo/cargo-run, r=alexcrichton

Just some minor test fixes.

Fix cargo-run rebasings and tests

Closes #443

Merge remote-tracking branch 'cmr/master'

Conflicts:
src/cargo/ops/cargo_run.rs
tests/test_cargo_run.rs

Fix building with current rust by using rust-lang/glob

The glob crate that comes with rust itself has been deprecated.

auto merge of #452 : alexcrichton/cargo/fix-nightly-linux, r=alexcrichton

The linux nightly failed and hopefully this will fix it.

Bump the git2 dep

The linux nightly failed and hopefully this will fix it.

auto merge of #446 : alexcrichton/cargo/libgit2-again, r=brson

I applied https://github.com/libgit2/libgit2/pull/2414 to a [local branch](https://github.com/alexcrichton/libgit2/tree/mingw-w64-winhttp) and then fixed the build for mingw-w64. The tests pass fine on mingw and I've used https successfully on windows with libgit2 on cargo now.

Roll back git2 a bit to fix tests

Update git2-rs with mingw support

An upstream PR for libgit2 was applied locally, modified to build, and then
git2-rs was updated to use this small for of libgit2. This adds https support
via winhttp on mingw for windows users.

Revert "Revert "Use libgit2 for driving git instead of the CLI""

This reverts commit 04440bb035082d4163f2211af447b8d67e077ccb.

Conflicts:
src/cargo/sources/git/utils.rs

auto merge of #444 : alexcrichton/cargo/issue-54, r=brson

This generates errors for malformed semver versions during the decoding process
rather than later in the convertion to a package id. This also cuts down on the
large number of derived traits to only what's necessary.

Closes #54

Disable travis on all branches but master

auto merge of #441 : EduardoBautista/cargo/fix-inconsistencies, r=alexcrichton

The guide appeared to be inconsistent with its use of "0.1.0" and "0.0.1" so I decided to change all version numbers to "0.0.1".

auto merge of #423 : alexcrichton/cargo/cargo-package, r=brson

This command will assemble the current package into a tarball ready for
uploading to the cargo registry. Currently no further verification is done
beyond packaging the local repository into a tarball, but in the future this
could execute other operations such as api stability tools.

Implement cargo-package

This command will assemble the current package into a tarball ready for
uploading to the cargo registry. Currently no further verification is done
beyond packaging the local repository into a tarball, but in the future this
could execute other operations such as api stability tools.

auto merge of #435 : alexcrichton/cargo/issue-431, r=brson

This option is used to disable the --test flag to rustc for a test or benchmark
target in order to signal that the binary already knows how to run the testing
infrastructure.

The test/benchmark is still compiled and run as usual, and the exit code is
expected to reflect the result of the test/benchmark.

Closes #431

Help debug a flaky windows test

Add a `harness` manifest option

This option is used to disable the --test flag to rustc for a test or benchmark
target in order to signal that the binary already knows how to run the testing
infrastructure.

The test/benchmark is still compiled and run as usual, and the exit code is
expected to reflect the result of the test/benchmark.

Closes #431

auto merge of #420 : alexcrichton/cargo/issue-393-2, r=brson

This commit removes all distributed executables except for `cargo`. All
builtin subcommands are implemented through library calls, and the fallback
methods are retained to maintain extensability through new subcommands.

Closes #393

Remove all subcommand executables

This commit removes all distributed executables except for `cargo`. All
builtin subcommands are implemented through library calls, and the fallback
methods are retained to maintain extensability through new subcommands.

Closes #393

auto merge of #421 : alexcrichton/cargo/issue-418, r=wycats

auto merge of #419 : alexcrichton/cargo/issue-406, r=brson

This allows tests to be built, but not run.

Closes #406

Fix `cargo run` for renamed bin targets

Add a --no-run option to `cargo test`

This allows tests to be built, but not run.

auto merge of #450 : alexcrichton/cargo/fix-cargo, r=alexcrichton

Fix a flaky test while the compiler is changing

Print out the rustc version before each build

Fix building with current rust

Decode directly into a semver::Version

This generates errors for malformed semver versions during the decoding process
rather than later in the convertion to a package id. This also cuts down on the
large number of derived traits to only what's necessary.

Closes #54

Use "0.0.1" instead of "0.1.0" for version numbers

cargo-run: support target and release the same as cargo-build

auto merge of #434 : alexcrichton/cargo/issue-433, r=huonw

All subprocesses will now be invoked with CARGO_MANIFEST_DIR pointing at the root of the
source directory that they are working on (compiling). This commit also
reorganizes the version environment variables to use the new infrastructure.

Closes #433